Update POTFILES.in and POTFILES.skip

SidebarRow: Add Unmount tooltip to eject_button

It was reported that the lack of a tooltip made its purpose unclear.
This can be solved by just copying PlacesViewRow’s eject_button tooltip.

https://bugzilla.gnome.org/show_bug.cgi?id=766909

GtkFileChooserNativeQuartz: restore parent focus after closing dialog

Tested for both modal and non-modal dialogs

https://bugzilla.gnome.org/show_bug.cgi?id=785306

GtkFileChooserNativeQuartz: improve support for file filters

Instead of using conditional compilation, use respondsToSelector to
check at runtime for setAccessoryViewDisclosed.

https://bugzilla.gnome.org/show_bug.cgi?id=785306

GtkFileChooserNativeQuartz: add support for Mountain Lion and Mavericks

https://bugzilla.gnome.org/show_bug.cgi?id=784723 introduced support for
native file chooser dialogs on macOS, but due to the use of generics in
the patch, there will be compilation errors on pre-Xcode 7 platforms,
such as Mountain Lion and Mavericks.

I strongly recommend to revert this patch when the oldest supported
macOS release is bumped to Yosemite (10.10).

https://bugzilla.gnome.org/show_bug.cgi?id=785306

gdk: wayland: always set buffer scale on size update

We're only updating this when we have an EGL window. This means we
have the wrong scaling factor when using Vulkan.

https://bugzilla.gnome.org/show_bug.cgi?id=786492

Update Friulian translation

Update Italian translation

Update Italian translation

getting_started: Sync filenames & section headings

The order of Examples 3 and 4 was swapped, so the headings updated
themselves, but the filenames were left reflecting the old order.

https://bugzilla.gnome.org/show_bug.cgi?id=786553

3.22.19

Handle emoji data change in emoji chooser

Update the settings schema to follow the change in Emoji data,
and make the emoji chooser code handle the new format.

Add shortnames to emoji data

The json file is from

https://github.com/emojione/emojione/tree/master/extras/alpha-codes

It is under the MIT license.

Don't consume too many key releases

The Emoji input support in GtkIMContextSimple was
unintentionally consuming too many key release
events. Fix this.

https://bugzilla.gnome.org/show_bug.cgi?id=786594

Update Spanish translation

Update Nepali translation

Update German translation

Update German translation

Update Nepali translation

Update Kazakh translation

Update French translation

Update French translation

Update German translation

Update Croatian translation

Update Croatian translation

gtkbutton: Avoid keyboard grabs

Seems to be there for the sole purpose of ensuring the button
shall receive the key release on keyboard-triggered activation.

For the cases where this makes sense (eg. comboboxes, menubuttons,
...) gtk+ already does ensure the menu is popup after key release.
This makes the grab pretty useless, and there's many other cases
where it doesn't make sense (eg. button being activated
programmatically from an event handler in another widget).

Fixes button activation unintendedly triggering shortcut inhibition
on wayland.

https://bugzilla.gnome.org/show_bug.cgi?id=786480

gdkwindow: Avoid re-setting the opaque region if it doesn't change

This avoids, at least, needless chatter with the compositor and the X
server in X11's case.

https://bugzilla.gnome.org/show_bug.cgi?id=786469

Update Slovak translation

Updated Czech translation

GDK-Win32: Fix AeroSnap on HiDPI

Commit 1d0fad3 revealed that there were some assumptions made that were
actually to compensate for the bug fixed by that commit, so we need to
remove those assumptions as they would result in AerSnap to not work
properly on HiDPI screens.

Also re-do how we set the x and y positions of our GdkWindow, so that we
are more consistent across the board when we go between a GDK window
coordinate and a Windows API window cooredinate.

This would also simplify the code a bit.

https://bugzilla.gnome.org/show_bug.cgi?id=785999

gdk/x11: Extract GdkDeviceTool tool ID from "Wacom Serial IDs" property

This property contains 5 integers, of which the last 2 respectively
contain the tool serial number and tool ID. We were only extracting the
first so far, but GdkDeviceTool also has API getters for the latter,
which remained 0.

https://bugzilla.gnome.org/show_bug.cgi?id=786400

Update Galician translation

GdkDisplay: Call the correct push|pop_error_trap()

It is wrong to assume all Displays are of the same class as the default.

https://bugzilla.gnome.org/show_bug.cgi?id=784016

ScrolledWindow: Bin unreachable ternary branches

We already checked that the child is not NULL above.

Coverity CID 1454191

Widget: Fix outdated comments about tooltips

They are not usually yellow anymore, the previous advice about how to
style them was for pre-3.20 versions, and the immediate replacement (CSS
class .tooltip) does not seem ready for primetime.

https://bugzilla.gnome.org/show_bug.cgi?id=784421

gtk/gtkemojichooser.c: Don't use g_autoptr()

We can't use g_autoptr() on non-GCC-style compilers.

https://bugzilla.gnome.org/show_bug.cgi?id=773299

Updated Lithuanian translation

Remove an erroneous unref

This was leftover by accident.

Redo emoji data slightly

No longer store variation sequences explicitly. Instead, put a 0
in the sequence where the modifiers will be inserted. This is more
compact, and it allows us to put variations directly into the
recent section. Update the type of the recent-emoji setting to
match these changes.

Don't update headings in an idle

It is not necessary since flow box invalidation is
all synchronous currently. If that ever changes,
this code will need adjustment.

testpopover: Sync initial vals of controls & props

The ComboBoxes were initially empty, rather than reflecting the initial
values of the properties. The CheckButtons were only correct by chance.
Fix this by setting the initial values on the widgets and binding them
to the properties using SYNC_CREATE, so the two are always synced up.

https://bugzilla.gnome.org/show_bug.cgi?id=786209

configure: Add missing quotes to unbreak an AS_IF

Without quoting, the comma in AC_MSG_ERROR was wrongly interpreted as
separating the if and else paths.

https://bugzilla.gnome.org/show_bug.cgi?id=786287

Update Spanish translation

Update Galician translation

Update Turkish translation

emoji: Fix a leak

We were leaking the list of recently used Emoji here.
Pointed out by Timm Bädert.

emoji: Start search on the first character

There is no need to wait for a longer prefix here.
We already display all the content anyway, it will
only get less.

Drop a redundant helper

We already had a function that does exactly what
is needed here.

Remove an unused field

Nothing uses bubble_window, so it can go.

Update Brazilian Portuguese translation

Update Brazilian Portuguese translation

InfoBar: Fix wrong/uninit var use in code example

https://bugzilla.gnome.org/show_bug.cgi?id=786144

Updated Serbian translation

Updated Serbian translation

Adwaita: Sync CSS files with current SASS

A recent commit for emoji also updated seemingly unrelated parts of the
generated CSS files, presumably due to other things that changed in
master. The CSS files should be kept in sync with their SASS sources.

LevelBar: Avoid unnecessarily copying allocation

Just use the original allocation, instead of copying it and passing that

Update Indonesian translation

Update Indonesian translation

convert-emoji: Add missing braces

convert-emoji: Don’t pointlessly do rest of loop

skip can never become FALSE again, so break as soon as it becomes TRUE.

Update Polish translation

emojichooser: Add context to the Objects category string

To disambiguate it from Objects in the Inspector.

Update POTFILES.skip

widget-factory: Add a use of show-emoji-icon

This also adds a testcase for nested popovers that was
missing in widget-factory so far.

entry: Add support for an Emoji chooser

Add an "Insert Emoji" item to the context menu in entries.
We also add a show-emoji-icon property, which when set to
TRUE, will add an icon that can be clicked to bring up
the Emoji chooser.

Fix a problem with focus handling in modal popovers

When the popover is dismissed, we return the focus to
where it came from. However, by using gtk_widget_grab_focus,
we were messing up the selection if that widget happens to
be an entry. Special-case GtkEntry and use
gtk_entry_grab_focus_without_selecting to avoid this issue.

Add an Emoji chooser widget

Add a popover that shows color Emoji, with a search entry.
The recently-used Emoji are stored in a GSetting.

Add Emoji data

The json file is imported from the (MIT-licensed) emoji.json[0] node
module, which generates it from the emoji list published by the
Unicode Consortium.

This commit also adds a little tool to convert the data into
a compact GVariant, and the result of that conversion, which is
added to libgtk as a resource. The following commits will make use
of it.

[0] https://github.com/amio/emoji.json

Update Polish translation

Update Galician translation

Update Galician translation

Update Friulian translation

Duplicate the TIC strings before NULLing the GObject data.

NULLing the data strings in GObject frees and invalidates the pointers
returned by g_object_get_data so we need our own copy of the values.

Revert "levelbar: Don't underallocate block widgets"

This reverts commit 78b4885fe8850e132d8bb06df8ab90ac6c2033e0.

That commit causes a block always to be shown at the min side, even when
the value is 0, in which case, no fill should be shown.

https://bugzilla.gnome.org/show_bug.cgi?id=783649

gtk/gtkfilechoosernativewin32.c: Declare variables at top-of-block

Since GTK+-3.22.x is not advertised to use C99, make sure that the build
works on pre-C99 compilers such as pre-2013 Visual Studio.

Avoid the use of g_autofree

We can't use this in code that may be built on win32,
so lets just not use it at all.

3.22.18

Layout: Fix documentation on getting the GdkWindow

Members are not public like they used to be, so accessors must be used.

Revert "Widget: Remove obsolete hack-arounds for HandleBox"

This reverts commit 12805a4fbfa66b10efbe48c03595b0d5b7608d0e.

I must have been looking at the wrong tree because HandleBox is only
deprecated in GTK+ 3, not removed.

Container: Don’t scroll to unset focus child coord

In gtk_container_real_set_focus_child(), we try to scroll to the
position of the new :focus-child if we have h or v adjustments.

gtk_widget_translate_coordinates() returns FALSE if neither widget is
realized or in other situations that cause output parameters x and y not
to be set. Thus, if the caller did not initialise x/y and uses them even
if the function returns FALSE, they are using uninitialised variables.

In gtk_container_real_set_focus_child(), we did not check the return
value but merrily went ahead and used x and y regardless. This is UB, as
caught by Valgrind, as well as being pointless.

The trivial fix is to exit early if (!gtk_widget_translate_coordinates).

https://bugzilla.gnome.org/show_bug.cgi?id=776909

Widget: Remove obsolete hack-arounds for HandleBox

It is long gone. The commit c9d9db0fcc8f29ab014c1e0b4b0b4f269f22f99c
identified these as ugly-but-functional workarounds for it, only.

https://bugzilla.gnome.org/show_bug.cgi?id=776909

main: Set the error if gtk_init_with_args fails

This fixes a fallout from 8a7d0ab481345708 where the error wasn't being
set when a display couldn't be opened right after parsing the
commandline.

It also fixes an older bug where the error would be left unset if the
commandline had already been parsed before (ie. when gtk_initialized
is TRUE).

https://bugzilla.gnome.org/show_bug.cgi?id=771959

docs: gtk_init_with_args will also fail if the commandline was invalid

Be more explicit to match the gtk_init_check documentation.

https://bugzilla.gnome.org/show_bug.cgi?id=771959

docs: gtk_init_check will also fail if the commandline was invalid

The existing documentation seems to suggest that gtk_init_check will
ignore any failure to parse the commandline arguments, and that its
return value only depends on its ability to initialize the windowing
system. That's not true.

Be more explicit to avoid misunderstandings.

https://bugzilla.gnome.org/show_bug.cgi?id=771959

AccelLabel: set_accel_closure() arg is nullable

AccelLabel: set_accel_widget() arg is nullable

Updated Slovenian translation

HeaderBar: Use less clashing appmenu fallback icon

process-stop-symbolic is unintuitive if represented as a stop sign as in
Adwaita, and completely ambiguous if represented as a cross like the
window close button in other icon themes.

Instead, use application-x-executable, which is already used elsewhere
as a fallback if no specific icon can be found for the application.

https://bugzilla.gnome.org/show_bug.cgi?id=784624

docs: gtk/running: Document the GTK_CSD env var

It was not documented before.

https://bugzilla.gnome.org/show_bug.cgi?id=778791#c1

AccelLabel: Fix typos of Ctl to Ctrl

icon-browser: Add missing document-edit icon

https://bugzilla.gnome.org/show_bug.cgi?id=777515

gdkdisplay: Remove a pointless assignment

Enough with the beeping!

Don't beep when modifiers are released in entries.
This was an inadvertent change that snuck in with
the emoji support.

Also, don't beep while entering an emoji name.
There is entirely too much beeping here.

AccelLabel: Fix displaying accel unichars >= 0x80

In GTK+ 2, the ch < 0x80 was ORd with klass->latin1_to_char, and that
was unconditionally set to TRUE in the class init function, so
effectively the ch < 0x80 never mattered before or served any purpose.

When klass->latin1_to_char was deleted from the class in commit
f760538f17673c5bd7fec792be2f1abf8148fc32, this check’s sense changed.

The resuls was that accel keyvals with gunichar value >= 0x80 stopped
being rendered as symbols, instead falling back to their keysym name.
Instead of recognisable symbols for these, we get raw, often obscure,
and untranslatable keysym names. This breaks accessibility as well as
client users who may be parsing such accels and migrating from GTK+ 2.

So, remove the < 0x80 to restore the behaviour from before said commit.

https://bugzilla.gnome.org/show_bug.cgi?id=783906

Add support for entering emoji by name

This commit adds some basic support for entering emoji by name
to GtkIMContextSimple. To begin an emoji sequence, use Ctrl-Shift-e
instead of Ctrl-Shift-u that is used for hex input. Otherwise, the
behavior is the same: you can can let go of the modifier keys and
end the sequence with space or enter, or hold on to the modifier
keys and end the sequence by releasing them.

Only a limited, fixed set of names is supported at this time, see
the GtkIMContextSimple docs for a full list.

levelbar: Don't underallocate block widgets

https://bugzilla.gnome.org/show_bug.cgi?id=783649

Window: Also quote Window address in Popover error

This is symmetrical with how we already do that for the Popover, and it
could be useful for debugging.

widget-factory: Add tooltip to Entry icon

just to show we can.

Fix some nullable Returns annotations

based on reports and patches by Iñaki García Etxebarria:

https://bugzilla.gnome.org/show_bug.cgi?id=781246
https://bugzilla.gnome.org/show_bug.cgi?id=785423

GtkFixed: Update documentation intro

• Add GtkLayout as a @See_also since it includes fixed-pos functionality
• Drop mention of the long-gone Linux framebuffer port
• Explain how to work around the problems with RTL text